Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[iOS] 1차 PR 수정 #35

Merged
merged 8 commits into from
Apr 23, 2021
Merged

[iOS] 1차 PR 수정 #35

merged 8 commits into from
Apr 23, 2021

Conversation

Settpark
Copy link
Collaborator

@Settpark Settpark commented Apr 23, 2021 •
edited by eeeesong
Loading

0422 PR의 피드백 반영하기

수정사항

  • SideDish: Class vs. Struct, var -> let
  • NetworkManager: Mock 객체 초기화 시에도 url 외부 주입, URLSession 부분 수정
  • MainViewController: bind를 private하게, Cold Observable 문제
  • MenuCellViewModel: print문 제거 -> 사용자에게 문제 상황 알리기
  • MainTableViewDataSource: cell 이름 static 상수로 변경
  • MainTableViewDelegate: headerView 분리하여 생성, 높이 값 상수로 저장
  • MainTableViewDataSource: ! 제거 -> 해당 항목은 백엔드 api로 바꾸면서 개선해보아요!

Settpark and others added 8 commits April 22, 2021 15:17
@Settpark
feat: Toast 추가
3bc2438
@Settpark
feat: Toast 기능 개발 : MenuCategory 부분
677c232
@eeeesong
feat: Toaster 기능 개발 Detail view 부분 (#10)
665dcc5
@eeeesong
feat: Toast UI 개선
6ba8fda 
- CustomToaster: 각 토스트의 생성 메소드 분리하여 UI 정교화
- Pods-ToastView: Bubble의 Alignment를 설정할 수 있도록 변경
@Settpark
fix : 피드백 반영 (#30)
ddf7564 
dquote>
dquote> class -> struct로 변경
dquote> 변수 var에서 let으로 변경
dquote> bind private 설정, Cold Observalbe 문제 해결
dquote> print문 제거 사용자에게 alertMessage 뜨도록 수정
@Settpark
fix : 피드백 반영 (#30)
36a32f3 
dquote>
dquote> colleciton에 해당하는 protocol선언 (임시)
dquote> TablecustomHeaderView 선언
@Settpark
Merge branch 'feature/Toast' of https://github.com/ghojeong/sidedish …
938fafb 
…into fix/firstPR
@eeeesong
fix: PR 피드백 반영 (#30)
c6808de 
- NetworkManager: Mock 객체 초기화 시에도 URL 외부 주입, 쓰지 않는 URLSession 삭제
- SideDish: CodingKey 추가
@Settpark Settpark added the ios 모바일 개발을 위한 태그 label Apr 23, 2021
@Settpark Settpark added this to the iOS App 개발하기 milestone Apr 23, 2021
@Settpark Settpark requested a review from eeeesong April 23, 2021 02:13
@eeeesong eeeesong merged commit 7c06ce1 into ios Apr 23, 2021
@Settpark Settpark linked an issue Apr 23, 2021 that may be closed by this pull request
[iOS] 1차 PR 수정 #30
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eeeesong eeeesong Awaiting requested review from eeeesong

Assignees

@Settpark Settpark

@eeeesong eeeesong

Labels
ios 모바일 개발을 위한 태그
Projects
None yet
Milestone
iOS App 개발하기
Development

Successfully merging this pull request may close these issues.

[iOS] 1차 PR 수정
2 participants
@Settpark @eeeesong